-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BetaProgramInitiator
deployment
#222
Conversation
Lynx:
NOTE: We may need to update the Tapir:
|
Fresh |
BetaProgramInitiator
deployment
We need to include these changes in this PR, right? And that would mandate a bump in |
Yes. One minor update to the ABI in the registry for the
From the
I suspect An update will occur eventually as part of 7.1.x, but I don't think it needs to be immediate because neither client nor server-side need it at the moment. |
It doesn't, but I still need to use the new Ok, I think I can implement it now without breaking anything / waiting for 7.1.x. |
Coordinator is proxied so you are still interacting with the same proxy contract, it's the underlying implementation contract that changed, and that change was an added method which is not used by the client/server libraries. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't approve because I opened it, but @derekpierre's commits look good to me.
Type of PR:
Required reviews:
What this does:
Issues fixed/closed:
Related to #188
Why it's needed:
Notes for reviewers: